Skip to content

[WIP] Add Datera storage plugin#3324

Closed
jcsdatera wants to merge 1 commit into
apache:4.11from
jcsdatera:4.11.datera
Closed

[WIP] Add Datera storage plugin#3324
jcsdatera wants to merge 1 commit into
apache:4.11from
jcsdatera:4.11.datera

Conversation

@jcsdatera
Copy link
Copy Markdown

@jcsdatera jcsdatera commented May 9, 2019

Features:

  1. Zone-wide and cluster-wide primary storage support
  2. VM template caching automatically on Datera, the subsequent VMs can be created instantaneously by fast cloning the root volume.
  3. Rapid storage-native snapshot
  4. Multiple managed primary storages can be created with a single Datera cluster to provide better management of
    • Total provisioned capacity
    • Default storage QoS values
    • Replica size ( 1 to 5 )
    • IP pool assignment for iSCSI target
    • Volume Placement ( hybrid, single_flash, all_flash )
  5. Volume snapshot to VM template
  6. Volume to VM template
  7. Volume size increase using service policy
  8. Volume QoS change using service policy
  9. Enabled KVM support
  10. New Datera app_instance name format to include ACS volume name
  11. VM live migration

Description

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to change)
  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • [*] Enhancement (improves an existing feature and functionality)
  • Cleanup (Code refactoring and cleanup, that may add test cases)

Screenshots (if appropriate):

How Has This Been Tested?

@jcsdatera
[WIP] Add Datera storage plugin
41e6989 
Features:

1. Zone-wide and cluster-wide primary storage support
2. VM template caching automatically on Datera, the subsequent VMs can be created instantaneously by fast cloning the root volume.
3. Rapid storage-native snapshot
4. Multiple managed primary storages can be created with a single Datera cluster to provide better management of
      - Total provisioned capacity
      - Default storage QoS values
      - Replica size ( 1 to 5 )
      - IP pool assignment for iSCSI target
      - Volume Placement ( hybrid, single_flash, all_flash )
 5. Volume snapshot to VM template
 6. Volume to VM template
 7. Volume size increase using service policy
 8. Volume QoS change using service policy
 9. Enabled KVM support
10. New Datera app_instance name format to include ACS volume name
11. VM live migration
@swill
Copy link
Copy Markdown
Contributor

swill commented May 10, 2019

I would like to add a little context to this PR and why it is opened against the 4.11 branch.

CloudOps originally developed this plugin for Datera, after which point, the Datera version and the CloudOps versions diverged as new functionality was added to both. This PR is an effort to consolidate the Datera and CloudOps implementations such that a mutual customer (who is running on the 4.11 LTS branch) may leverage the functionality.

Here is the plan for release:

  • CloudOps will create a PR on this PR to add the functionality that we have developed on top of this.
  • Once the combined functionality has been validated, our mutual customer will pull the working PR on the 4.11 branch into their code base.
  • This PR will then be rebased against the master branch for formal inclusion in the upstream ACS code base.
  • Datera will maintain this plugin in the upstream code base going forward.

Let me know if you have any questions or concerns and I will work to facilitate the inclusion of this code upstream.

@yadvr
Copy link
Copy Markdown
Member

yadvr commented May 16, 2019

@swill thanks for sharing and being open about it. I was going to kick tests on the PR, I guess I'll wait until this gets re-posted against master branch.

@yadvr yadvr added the On Hold label May 27, 2019
@swill
Copy link
Copy Markdown
Contributor

swill commented May 27, 2019

@rhtyd thank you. We will get this PR updated with the additional functionality soon. Cheers...

@yadvr
Copy link
Copy Markdown
Member

yadvr commented Jun 20, 2019

Can we close this @jcsdatera @swill in favour of #3410 ?

@yadvr
Copy link
Copy Markdown
Member

yadvr commented Jun 26, 2019

Closing this in favour of #3410 new PR against master

@yadvr yadvr closed this Jun 26, 2019
@jcsdatera
Copy link
Copy Markdown
Author

jcsdatera commented Jul 9, 2019

Ping @rhtyd, @swill
Could we reopen this PR as we are still waiting for CloudOps to add functionality on top of this?

Thank you

@yadvr yadvr reopened this Jul 9, 2019
@yadvr
Copy link
Copy Markdown
Member

yadvr commented Jul 9, 2019

@jcsdatera we cannot accept changes towards 4.11 branch, kindly submit a new PR for master, or use an existing PR against master for adding functionality.

@yadvr yadvr closed this Jul 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

component:primary-storage status:work-in-progress type:enhancement

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@jcsdatera @swill @yadvr @svenvogel